Skip to content

docs(ai): update claude testing rules to reflect NSubstitute over Moq#1216

Open
cesarcastrocuba wants to merge 1 commit into
fullstackhero:mainfrom
cesarcastrocuba:pr/update-ai-testing-docs
Open

docs(ai): update claude testing rules to reflect NSubstitute over Moq#1216
cesarcastrocuba wants to merge 1 commit into
fullstackhero:mainfrom
cesarcastrocuba:pr/update-ai-testing-docs

Conversation

@cesarcastrocuba
Copy link
Copy Markdown
Contributor

Updates the AI instruction files to rely on NSubstitute instead of Moq, aligning with the project's testing standards.

@maxiar
Copy link
Copy Markdown
Contributor

maxiar commented Mar 11, 2026

@iammukeshm Please approve it, I already use it and works fine.

@iammukeshm iammukeshm deleted the branch fullstackhero:main May 27, 2026 06:59
@iammukeshm iammukeshm closed this May 27, 2026
@iammukeshm iammukeshm reopened this May 27, 2026
@iammukeshm iammukeshm changed the base branch from develop to main May 27, 2026 07:01
@iammukeshm
Copy link
Copy Markdown
Member

Thanks for the substantial work across these PRs 🙏

FullStackHero has moved to a major new baseline — v10 (.NET 10) — which includes a significant restructure (for example src/api/framework/** is now src/BuildingBlocks/**) and a new CI pipeline. This PR now targets main but no longer applies cleanly against it.

Could you please:

  1. Rebase your branch onto the current main and resolve conflicts,
  2. re-review your changes against the v10 layout/APIs (some files have moved or changed), and
  3. push the update so the PR is green against the new Backend CI / Frontend CI checks.

Once it's rebased and passing we'll be happy to review. Really appreciate the contributions to FullStackHero!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants